Skip to content

Conversation

oliver-sanders
Copy link
Member

@oliver-sanders oliver-sanders commented Sep 4, 2025

Improve broadcast documentation and cover the broadcast expiry change in cylc/cylc-flow#6964

Requirements check-list

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.

@oliver-sanders oliver-sanders added this to the 8.6.x milestone Sep 4, 2025
@oliver-sanders oliver-sanders self-assigned this Sep 4, 2025
@oliver-sanders oliver-sanders marked this pull request as ready for review September 4, 2025 16:00
Copy link
Member

@hjoliver hjoliver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good, a few minor changes suggested.

Copy link
Member

@hjoliver hjoliver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few more tweaks suggested.

@MetRonnie MetRonnie self-requested a review September 12, 2025 13:06
Co-authored-by: Hilary James Oliver <[email protected]>
Copy link
Member

@hjoliver hjoliver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approval subject to two tweaks.

@wxtim wxtim requested review from wxtim and removed request for MetRonnie September 19, 2025 11:33
Copy link
Member

@wxtim wxtim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alles Gut. Please take @hjoliver and my suggestions.

Co-authored-by: Hilary James Oliver <[email protected]>
Co-authored-by: Tim Pillinger <[email protected]>
==============

Cylc "Broadcasts" allow us to override the :cylc:conf:`[runtime]` settings of tasks.
settings within a running workflow.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
settings within a running workflow.

Copy link
Member

@hjoliver hjoliver Sep 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Kinda my fault - I'll push the change and merge)

@hjoliver hjoliver merged commit cf5199f into cylc:master Sep 20, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants